Skip to content

Conversation

@legal90
Copy link
Contributor

@legal90 legal90 commented Jan 20, 2025

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

  • N
  • Y

2. What is the scope of this PR (e.g. component or file name):

docs/user_docs/guides/working-with-konfig/*

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

This PRs updates code examples and snippets according to the latest changes in kcl-lang/konfig repo, especially this: kcl-lang/konfig#31

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

  • N
  • Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

  • Unit test
  • Integration test
  • Benchmark (add benchmark stats below)
  • Manual test (add detailed scripts or steps below)
  • Other

@vercel
Copy link

vercel bot commented Jan 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kcl-lang-io ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 20, 2025 4:20pm

Signed-off-by: Mikhail Zholobov <legal90@gmail.com>
Signed-off-by: Mikhail Zholobov <legal90@gmail.com>
Signed-off-by: Mikhail Zholobov <legal90@gmail.com>
Copy link
Contributor

@Peefy Peefy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! LGTM!

@Peefy Peefy merged commit 1a8161b into kcl-lang:main Jan 20, 2025
5 of 7 checks passed
@legal90
Copy link
Contributor Author

legal90 commented Apr 16, 2025

@Peefy, thanks for merging this PR!
I see it was supposed to be released in 0.11 (https://github.com/kcl-lang/kcl-lang.io/releases/tag/v0.11),

But the actual rendered version of the website still contains the old data: https://www.kcl-lang.io/docs/user_docs/guides/working-with-konfig/guide

Though, the "next" version is shown correctly, with this fix included: https://www.kcl-lang.io/docs/next/user_docs/guides/working-with-konfig/guide

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants